Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

合并文件时，优先从内存中读取（匿名模块），否则才同步IO操作（命名模块） #101

Open
wants to merge 1 commit into
base: 1.0/develop
Choose a base branch
from
Open

合并文件时，优先从内存中读取（匿名模块），否则才同步IO操作（命名模块） #101

wants to merge 1 commit into from

Conversation

wenber
Copy link

@wenber wenber commented Oct 15, 2015

No description provided.

@leeight
Copy link
Member

leeight commented Oct 15, 2015

Can one of the admins verify this patch?

@wenber
Copy link
Author

wenber commented Oct 15, 2015

目前文件合并全部是采用同步读取文件，实质上可以直接使用内容中的数据，但是我测试了，命名模块名不是有效的processContext对象的哈希值，此时需要读取文件，建议processContext的哈希值采用绝对路径，就可以避免这个命名模块的路径问题

@wenber
Copy link
Author

wenber commented Oct 22, 2015

请你们测试下，目前我们需要这个优化来提高js文件合并操作的速度，如有问题，请指出，谢谢哈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wenber @leeight